Skip to content

refactor(mode-home): tighten hero subtitles to a single phone line#976

Merged
BigSimmo merged 7 commits into
mainfrom
claude/factsheet-copy-polish-pkdxz8
Jul 20, 2026
Merged

refactor(mode-home): tighten hero subtitles to a single phone line#976
BigSimmo merged 7 commits into
mainfrom
claude/factsheet-copy-polish-pkdxz8

Conversation

@BigSimmo

@BigSimmo BigSimmo commented Jul 20, 2026

Copy link
Copy Markdown
Owner

Summary

  • Follow-up to refactor(mode-home): short titles and single-line hero subtitles #966: shorten every mode-home hero subtitle to ~30–40 characters so it renders on one line at phone widths.
  • Examples: Factsheets → "Plain-language patient handouts.", Tools → "Assessment, prescribing, workflows.", Forms → "The WA MHA 2014 forms register.", Therapy → "{n} source-grounded therapy records.", Medication → "Medication dosing and safety.", DSM-5 → "Criteria, specifiers, and comparisons.", Specifiers → "Check specifier fit and exclusions.", Formulation → "Build a formulation from the evidence.", Services → "Search by need, catchment, or route.", Differentials → "Match your catalogue to your library.", Favourites → "Saved notes, sources, and sets."
  • Copy-only change to the subtitle props on ModeHomeHero/ModeHomeTemplate; no structure or CSS change. Titles are unchanged from refactor(mode-home): short titles and single-line hero subtitles #966.

Verification

  • Rendering measured in Chromium at 320 / 360 / 390px viewports: every mode-home subtitle occupies exactly one line box (counted via Range.getClientRects() against the running app). Favourites is auth-gated so its hero didn't render headlessly, but at 31 chars it's shorter than lines that already fit at 320px.
  • npm run typecheck ✅, npm run lint ✅, vitest run tests/therapy-compass-data-recovery.dom.test.tsx ✅ (the therapy hero subtitle is dynamic, so its DOM-test regexes were updated to the shortened copy).
  • UI verification (verify:ui) runs in CI — this sandbox's Playwright browser build is mismatched, so the affected specs are validated there.

Risk and rollout

Clinical Governance Preflight

Not applicable — no ingestion, answer generation, search/ranking, source rendering, document access, privacy, production env, or clinical-output surface is touched. Presentational hero copy only.

Notes

🤖 Generated with Claude Code


Generated by Claude Code

Summary by CodeRabbit

  • Style
    • Refined homepage subtitles across clinical, diagnostic, medication, forms, formulation, services, and other sections for clearer, more concise wording.
    • Simplified therapy resource count messaging by removing references to clinical pathways while preserving accurate singular and plural counts.
  • Tests
    • Updated therapy recovery checks to reflect the revised wording and verify accurate recovered record counts.

Shorten every mode-home subtitle to ~30-40 characters so it renders on
one line at phone widths. Verified in Chromium at 320/360/390px that each
subtitle occupies a single line box.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Gb9jucaNMEQDmRKXEN7SRV
@supabase

supabase Bot commented Jul 20, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project sjrfecxgysukkwxsowpy because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@coderabbitai

coderabbitai Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: fa083887-4f43-4a4e-a81b-63603c5eae80

📥 Commits

Reviewing files that changed from the base of the PR and between 63b1a91 and f35cb21.

📒 Files selected for processing (12)
  • src/components/applications-launcher-page.tsx
  • src/components/clinical-dashboard/differentials-home.tsx
  • src/components/clinical-dashboard/favourites-hub.tsx
  • src/components/clinical-dashboard/medication-prescribing-workspace.tsx
  • src/components/dsm/dsm-home-page.tsx
  • src/components/factsheets/factsheets-home-page.tsx
  • src/components/forms/forms-home-page.tsx
  • src/components/formulation/formulation-home-page.tsx
  • src/components/services/services-home-page.tsx
  • src/components/specifiers/specifiers-home-page.tsx
  • src/components/therapy-compass/screens/home-screen.tsx
  • tests/therapy-compass-data-recovery.dom.test.tsx

📝 Walkthrough

Walkthrough

Homepage subtitles and Therapy Compass record-count wording were revised. Therapy Compass recovery tests were updated to match the new singular and plural text.

Changes

UI copy refinement

Layer / File(s) Summary
Homepage subtitle updates
src/components/applications-launcher-page.tsx, src/components/clinical-dashboard/*, src/components/dsm/*, src/components/factsheets/*, src/components/forms/*, src/components/formulation/*, src/components/services/*, src/components/specifiers/*
Revised static subtitle and description text across the listed homepages.
Therapy Compass copy and recovery assertions
src/components/therapy-compass/screens/home-screen.tsx, tests/therapy-compass-data-recovery.dom.test.tsx
Removed “clinical pathways” from the loaded count copy and updated recovery assertions for the revised record messages.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: shortening mode-home subtitles for single-line phone rendering.
Description check ✅ Passed The description covers summary, verification, risk/rollback, governance, and notes, matching the template well.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/factsheet-copy-polish-pkdxz8

Comment @coderabbitai help to get the list of available commands.

@BigSimmo
BigSimmo marked this pull request as ready for review July 20, 2026 09:10
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@BigSimmo
BigSimmo enabled auto-merge (squash) July 20, 2026 09:10
@BigSimmo
BigSimmo merged commit e727ed3 into main Jul 20, 2026
17 checks passed
@BigSimmo
BigSimmo deleted the claude/factsheet-copy-polish-pkdxz8 branch July 20, 2026 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants